-
-
Notifications
You must be signed in to change notification settings - Fork 833
Warn when migrating e2e data to indexeddb #1654
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there any way we can avoid adding another 33 lines to a 1600 line file?
"This will have caused end-to-end cryptography to malfunction "+ | ||
"in the older version. End-to-end encrypted messages exchanged "+ | ||
"recently whilst using the older version may not be decryptable "+ | ||
"in this version.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
... and nor may messages exchanged in future, because people think you have different device keys. log out, export keys, etc etc
sigh. I can't think of a way. |
also a comma
Mm, we could probably think about having a class that sits & listens for general things like this and pops up dialogs. It would still have to be started & stopped by MatrixClientPeg though. Either way, probably one for a refactor down the line. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
presumably needs an update to en_EN.json?
lgtm otherwise
Twinned with matrix-org/matrix-js-sdk#586